-
-
Notifications
You must be signed in to change notification settings - Fork 135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TASK: Use saucectl with configuration #3851
Conversation
c713805
to
686c4af
Compare
Also configure a tunnel.
99c4e39
to
22bb4af
Compare
The validation of the saucelabs config failed
Resolves: ERR failed to execute run command error="'node_modules' is ignored by sauceignore, but you have npm dependencies defined in your project; please remove 'node_modules' from your sauceignore file"
Resolves: sauce_connect proxying is denied to host "127.0.0.1:8081" localhost proxying is disabled
1d98fd3
to
fa168a0
Compare
e7f9d88
to
dbf89a2
Compare
As the resolution we specified for Mac is not available.
Removes the post-acceptance-tests-recordings in CircleCI and the related script.
7d00005
to
de6bce0
Compare
3333094
to
6d0e10e
Compare
e05b62c
to
f038db6
Compare
f038db6
to
863412f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks so much <3 <3
@@ -108,7 +108,7 @@ test: | |||
|
|||
## Executes integration tests on saucelabs. | |||
test-e2e-saucelabs: | |||
bash Tests/IntegrationTests/e2e.sh "saucelabs:Firefox@latest:Windows 10" | |||
bash Tests/IntegrationTests/e2e-saucelabs.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do we need another script that is almost e2e.sh
? that will it make hard to maintain, especially since in neos 9 the script is partly rewritten. So we def need adjustments for Neos 9 ... and it probably better to have a shared script for local and sauce e2e tests?
TASK: Use saucectl with configuration
Overview:
We are using sauce labs as a remote browser service. The versions we were using are on an API version that has been deprecated. So this Pull Request brings the sauce labs usage to the latest versions. For that, we are now using the CLI tool of sauce labs
saucectl
. To be able to access the remote browsers,saucectl
needs on CircleCI a tunnel. So we also need to use theSauce Connect Proxy 5 CLI
.Changes:
Use
saucectl
CLI Tool:saucectl
CLI for managing and executing Sauce Labs tests.saucectl
runs thetestcafe
testsRemoval of Sauce Labs Test Links in PR Comments:
New Features Enabled:
New view of tests:
Old view of tests:
Future Improvements:
With these updates, we have laid the groundwork for: